Pull Request Test Coverage Report for Build 18244744117

Details

• 68 of 152 (44.74%) changed or added relevant lines in 9 files are covered.
• 3 unchanged lines in 2 files lost coverage.
• Overall coverage increased (+3.0%) to 30.759%

Totals
Change from base Build 18143162569:	3.0%
Covered Lines:	1326
Relevant Lines:	4311

💛 - Coveralls

@Norbytus I will have a more detailed look at this this weekend and might do some tinkering myself. Is this up to date? Might push some changes directly to this branch.

@Norbytus I will have a more detailed look at this this weekend and might do some tinkering myself. Is this up to date? Might push some changes directly to this branch.

You can to this branch, it's ok.
Or just can write some comment.
I still thinking about should we have separate builder for interfaces and how make optional construct.

Hi, this looks like there was a lot of work done, great job! Any chance this would be completed and merged? I'm thinking about using ext-php-rs for my extension and this would simplify the implementation a lot.

@jakubkulhan had to take some time off, but will push this now that I have more time. There is some work left, but should be done soon.

@Norbytus could you rebase onto master? I can't do this because workflow files changed and those need extra permissions to update.

@Norbytus could you rebase onto master? I can't do this because workflow files changed and those need extra permissions to update.

Done

@Norbytus will try to do some refactoring on the derives and have some minor code style adjustments in the works. Ill also add some more tests. Other than that it looks good to me. Thank you so much for your work and sorry again for the huge delay getting to this.

@Norbytus will try to do some refactoring on the derives and have some minor code style adjustments in the works. Ill also add some more tests. Other than that it looks good to me. Thank you so much for your work and sorry again for the huge delay getting to this.

No problem, i will check tomorrow